Skip to content

docs: clarify Azure project URL support - #2593

Draft
roji wants to merge 1 commit into
mainfrom
roji-azure-project-url-docs
Draft

docs: clarify Azure project URL support#2593
roji wants to merge 1 commit into
mainfrom
roji-azure-project-url-docs

Conversation

@roji

@roji roji commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

The BYOK guidance currently requires host-only Azure URLs, excluding project-scoped endpoints. Update the existing Azure notes to describe host or project URLs with optional trailing slashes, show the resulting versionless Responses URL, and note the updated CLI runtime requirement.

Documentation only; no SDK behavior changes.

Related to #2588.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 9, 2026 16:43
@roji
roji requested a review from a team as a code owner September 9, 2026 16:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The minimum compatible Copilot CLI version must be specified.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 1 Low severity

New issues introduced by this change (1)
Severity Finding
Low severity docs/​auth/​byok.md — State the minimum Copilot CLI version that supports project URLs instead of saying only “updated.”…
What changed in this PR

Clarifies Azure project-scoped URL support in BYOK documentation.

Changes:

  • Documents host and project URL formats.
  • Shows versionless Responses URL construction.
  • Notes the Copilot CLI runtime dependency.
File Description
docs/​auth/​byok.md Updates Azure endpoint guidance and example comments.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/auth/byok.md
@SteveSandersonMS
SteveSandersonMS marked this pull request as draft September 10, 2026 14:40
@SteveSandersonMS

Copy link
Copy Markdown
Contributor

I manually verified this against the SDK-bundled Copilot CLI v1.0.83 and the latest published preview, v1.0.84-3. In both cases, an Azure baseUrl ending in /api/projects/hosted-agents-ncus produced a request to /openai/v1/responses, dropping the project prefix. The documentation therefore depends on a runtime update that is not yet available and verified.\n\nFor tracking, I have moved this PR to draft pending that runtime update. Please mark it ready for review once a runtime containing the fix is available and the final outbound URL has been verified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants